-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add aerosol optical depth to AerosolState #567
Conversation
ee1ddce
to
6566a69
Compare
Yes, this looks like the right direction. We only need the vertical sum of AOD though, not every layer. And I think if you could output AOD in this test: https://github.com/CliMA/RRTMGP.jl/blob/main/test/all_sky_with_aerosols_utils.jl, it would be a good step towards having it in ClimaAtmos. |
@sriharshakandala, can you please take a look at this? |
pseudocode
|
2ce75b0
to
65f2ce1
Compare
13a0467
to
28f12c8
Compare
I'm fine with incrementing the minor version in this PR, too. cc @szy21 |
Oh, right, I forgot to do it. |
This PR adds aerosol optical depth to AerosolState.
@szy21, can you please let me know if this looks like I'm on the right track, or if there are changes needed here?